Skip to content

doc(linstor): fix wrong comments on _call_local_method - #180

Merged
Wescoeur merged 1 commit into
3.2.12-8.3-develfrom
agf-linstor_call_retry_on_ENODATA_ENOENT
Oct 7, 2026
Merged

Wescoeur merged 1 commit into
3.2.12-8.3-develfrom
agf-linstor_call_retry_on_ENODATA_ENOENT

Conversation

@Ythogtha

@Ythogtha Ythogtha commented Oct 5, 2026 •

Copy link
Copy Markdown

util.retry(local_call) states that it should stop retrying on EROFS Exception.
The actual code will do the same for ENODATA or ENOENT.
Though ENODATA could be transient on DRBD, therefore being a good reason for retrying.

ENODATA is retried directly on blktap2.

The comment was out-of-date, this PR fixes it, avoiding future confusion.

@Ythogtha
Ythogtha force-pushed the agf-linstor_call_retry_on_ENODATA_ENOENT branch from 2be6216 to 07bd954 Compare October 5, 2026 15:44
@Ythogtha
Ythogtha requested a review from a team October 5, 2026 15:44
@Ythogtha
Ythogtha force-pushed the agf-linstor_call_retry_on_ENODATA_ENOENT branch from 07bd954 to cc0ef59 Compare October 5, 2026 15:47
@Ythogtha
Ythogtha marked this pull request as draft October 5, 2026 16:23
@Ythogtha
Ythogtha force-pushed the agf-linstor_call_retry_on_ENODATA_ENOENT branch from cc0ef59 to f9d6b6f Compare October 6, 2026 07:32
@Ythogtha Ythogtha changed the title fix(LINSTOR) local_call retry possibly transient ENODATA error on DRBD. doc(LINSTOR) Fix comments on _call_local_method. Oct 6, 2026
@Ythogtha
Ythogtha marked this pull request as ready for review October 6, 2026 09:10
@Ythogtha
Ythogtha force-pushed the agf-linstor_call_retry_on_ENODATA_ENOENT branch from f9d6b6f to 0bd8521 Compare October 6, 2026 09:13
Signed-off-by: Arnaud Garcia-Fernandez <arnaud.garcia-fernandez@vates.tech>
@Ythogtha Ythogtha changed the title doc(LINSTOR) Fix comments on _call_local_method. doc(linstor) Fix comments on _call_local_method. Oct 6, 2026
@Wescoeur Wescoeur changed the title doc(linstor) Fix comments on _call_local_method. doc(linstor): fix wrong comments on _call_local_method Oct 6, 2026
@Wescoeur
Wescoeur requested a review from a team October 6, 2026 09:19
@Wescoeur
Wescoeur merged commit c595dca into 3.2.12-8.3-devel Oct 7, 2026
5 checks passed
@Wescoeur
Wescoeur deleted the agf-linstor_call_retry_on_ENODATA_ENOENT branch October 7, 2026 14:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants